Merge branch 'gtk-3-24' into 'gtk-3-24'

GtkWindow: Handle non-square icons with height > width correctly

See merge request GNOME/gtk!173

GtkWindow: Handle non-square icons with height > width correctly

menu: Remove unstable annotations

These apis are not going to change in GTK+ 3.x.

Drop Ctrl-Shift-e support

Now that we have Emoji completion, drop the rather limited
Ctrl-Shift-e support in GtkIMContextSimple, and leave this
sequence to input methods.

widget-factory: add emoji completion to an entry

entry: Add emoji completion

Pop up completions when the text in the entry matches :word:
This functionality has to be enabled using the enable-emoji-completion
property.

Add an emoji completion popup

This widget provides entry completion-like functionality
for Emoji codes like :grin: or :kiss:.

Export some entry functions privately

This will be used in the following commits.

emoji: Hide recent section when empty

This is part of https://gitlab.gnome.org/GNOME/gtk/merge_requests/143
by Julian Sperber.

emoji: Improve section scrolling

Leave some space above the section heading when scrolling.

This is a part of https://gitlab.gnome.org/GNOME/gtk/merge_requests/143
by Julian Sperber.

Update Catalan translation

treeview: respect expander-size style property

Without enforcement to the expander-size, we can end up rendering icons
rather fuzzy. This uses the expander-size style property to determine
the square for the icon, centered on what was the calculated space for
the expander.

widget: Fix example code in doc

gtksocket: Adjust X sizes by scale-factor

X uses unscaled sizes, so they must be scaled properly. Otherwise
GtkSockets end up twice as big as they should be.

Closes: https://bugzilla.gnome.org/show_bug.cgi?id=765327
Signed-off-by: Jason Zaman <jason@perfinion.com>
Close !165

Merge branch 'gesture-docs' into 'master'

Gesture stuff

See merge request GNOME/gtk!169

(cherry picked from commit 8b24d59cde9410db28c7b61d323fa8b1f0180300)

334c7911 gesture: Fix get_last_event() docs
e9765c04 gesture: Fix code snippet
4d2b39d9 gesturemultipress: Don’t fire ::released after ::cancel

x11: Don't set NET_WM_PID when sandboxed

It is not useful, and some window managers misinterpret it and
add some "runs as root" indication to the window decoration.

See https://github.com/mate-desktop/marco/issues/301

gdk: Add a private api to find sandboxes

This will be used in more places in the future.

Update Spanish translation

gdkdnd: Fix 2 typos in docs

ci: update the docker image to fedora 28

So we get a newer pango

Merge branch 'gtk-3-24' into 'gtk-3-24'

icontheme: Keep dir_mtimes in order

See merge request GNOME/gtk!162

Update Polish translation

Update POTFILES.in and POTFILES.skip

SpinButton: Fix an obviously wrong arg description

:climb-rate is not about what you get when you single-click on a button,
as this implied: it's what happens if you hold down a button or a key.
Fix the description of @climb_rate to new(), and while here, mention the
key in the blurb of :climb-rate itself.

GdkWin: Avoid leak if bailing from process_updates

There is no point creating the list if we don't need to because we are
failing out, and even less point in leaking it in that case.

StyleContext: Add a missing apostrophe

(or rather, the right single quote dictated by the HIG, which is
semantically not an apostrophe at all! I will die on this hill.)

wayland: check native window for crossing events

gdk_wayland_*_grab()/ungrab() would emit crossing events which translate
as focus_in/focus_out events for keyboard.

However, the ungrab() functions compare the native toplevel as this is
what gets the Wayland pointer enter/leave events with the grab window,
so if the grab is issued on a child gdk window, those won't match and we
would emit more focus_out events than focus_in events.

This means that a widget such as spice-gtk which issues a keyboard grab
whenever the pointer enters the window and releases the grab when it
leaves the window would get uneven numbers of focus_in/focus_out events.

Also, gdk_wayland_seat_ungrab() would not emit crossing events for
keyboard devices, whereas gdk_wayland_device_ungrab() does, which adds
even more potential discrepancies between focus_in/focus_out events.

To solve this problem, introduce two new helper functions which check
the relevant native windows to emit crossing events when needed that get
called evenly from both gdk_wayland_seat_grab()/ungrab() and gdk_Wayland
_device_grab()/ungrab() APIs.

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=780422
Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/792

icontheme: Keep dir_mtimes in order

Don't reverse the order each time we insert a theme. Reverse it only
once, after all themes have been loaded.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1115

StyleContext: Don't advise using invalidate()

It's deprecated. Also, set_screen() is not used/needed in the demo cited

Remove the "without parent" warning

We've had it for a long time, and it hasn't really made
a difference. And I don't think we are prepared to turn
this into a hard error. So just drop it.

Range: Up should only mean ++ if we are a GtkScale

The last round of patches to get the desired direction of value move in
response to scrolls/keypresses on scales had the inadvertent side effect
of giving the opposite direction on scrollbars. Seeing as gtkrange.c is
already a collection of hacks, add another so that fix only holds if the
instance is a GtkScale, since that is what those patches were aimed at.

Close https://gitlab.gnome.org/GNOME/gtk/issues/1065

ProgressBar: Fix typo in doc comment

font chooser: Support font variations

Add sliders for the axes on the tweak page.

font chooser: Add properties for font features and language

These can't be returned as part of the font description,
so we need new api for them. For now, this is just readonly
properties. Maybe these should be writable too, eventually.

font chooser: Add font features

Populate the tweak page with OpenType font features.

font chooser: Add a tweak page

font chooser: introduce font chooser levels

This is a more systematic approach to reducing the
level of detail in the font chooser.

Bump the pango requirement

We want to use the new font variation api in pango 1.41.

Add a flatpak manifest for widget-factory

This makes it easier to build GTK+.

Merge branch '1053-scroll-cursor-gets-left-behind-if-a-child-widget-steals-the-scroll' into 'gtk-3-22'

Resolve "Scroll cursor gets left behind if a child widget steals the scroll"

See merge request GNOME/gtk!134

textview: Don't scroll for pastes in another view

GtkTextView scrolls to the insertion point when the text
buffer signals a paste is done. This is wrong when there
are multiple views on the same buffer, and the paste
happened in another view.

To fix this, flip the handling of the scroll_after_paste
boolean to only be TRUE if we know that we want to scroll.

Merge branch '1069-thread-1-glade-received-signal-sigfpe-arithmetic-exception' into 'gtk-3-22'

Resolve "Thread 1 "glade" received signal SIGFPE, Arithmetic exception."

See merge request GNOME/gtk!151

appchooserdialog: Fix memory leak

appchooserdialog: Don't fire notify::heading twice

The gtk_app_chooser_dialog_set_heading() function do emit
notify::heading. Since the setter simply calls the function,
the setter itself shouldn't emit a notify signal by itself.

textview: Always show select-all button in touch popup

"Select all" action is possible regardless of the text selection state
or if the text is editable or not.

textview: Simplify creating bubble action buttons

Buttons can be created simply with gtk_button_new_from_icon_name().

GtkBox: do not divide by zero on gtk_box_size_allocate_with_center()

https://gitlab.gnome.org/GNOME/gtk/issues/1069

Visual Studio 2008 builds: Generate gtk/gtktypefuncs.c

This is the updates to the Visual Studio 2008 projects to generate
gtk/gtktypefuncs.c using the preprocessor and the gentypefuncs.py that
was adapted from master.

Unfortunately we could not clean up the projects as we did for the 201x
ones due to the differences in project file format.

Visual Studio 201x builds: Clean up project files

Combine repetitive parts, and unify using $(PythonDir) for all builds,
which the paths set in the property sheets are now based on the Visual
Studio version and platform combination.

Visual Studio 201x builds: Fix previous commit

Not that it will make a difference, but to be consistent with the
autotools builds.  Include gtkx.h instead of gtk.h when we generate the
source to feed to the preprocessor.

Visual Studio 201x builds: Generate gtk/gtktypefuncs.c

We now need to generate gtktypefuncs.c by ourselves, so modify the
gentypefuncs.py script from master, and add a custom build step in the
projects to generate gtktypefuncs.c.  The custom build step for the 2008
projects will be added later.

Merge branch 'wip/restart-cursor-animation-gtk-3' into 'gtk-3-22'

wayland: Fix restarting cursor animation

See merge request GNOME/gtk!150

gtk/Makefile.am: Fix Visual Studio project generation

We now need the generated sources to be included explicitly in the list
so that they get listed in the project files to be built.

wayland: Fix restarting cursor animation

When an animated cursor was set and the previous cursor animation delay
happened to be the same, we wouldn't restart the animation timeout and
just return G_SOURCE_CONTINUE assuming the timer would continue. This
assumption is however only valid if the function was called from the
timeout, which is not the case.

Instead also arm the timer also if there is no previous timer active.

[Quartz] Hardcode screen resolution for text at 96.0.

It seems that CoreText is internally calibrated for this, see
https://bugzilla.gnome.org/show_bug.cgi?id=787867 for a detailed
discussion.

gtk: Fix reserved pointers

So gcc stops complaining about unnecessary parenthesis.

Closes #1059

Range: Bin pointless check before emitting signal

In scroll_event(), there is no need to check whether we are realized
before emitting ::change-value, as we must be when receiving an event.

Git-formatted/rebased/cleaned up by Daniel Boles <dboles.src@gmail.com>

Close https://gitlab.gnome.org/GNOME/gtk/issues/292

Menu: cleanups for previous commit and nearby

• #include <math.h> for the new uses of floor()
• Move the new ints and popdown_data into the scopes where they are used
• Don’t pointlessly init other ints to 0 as they always get reassigned
• Burninate gint

Menu: Fix broken navigation triangle/hysteresis

This issue was caused when mouse coordinates were changed to floating
point values in commit e8b38fedbd8961df4aaaa75b122a06ddc68e75f2.

This patch floors the event->x_root and event->y_root values when
setting the navigation region, so the previous behaviour is restored.

https://gitlab.gnome.org/GNOME/gtk/issues/450

ScrolledWindow: Unset cursor if child takes scroll

https://gitlab.gnome.org/GNOME/gtk/issues/1053

x11: Set a transparent background on windows by default

This avoids black flicker on compositing WMs when a window is first shown.

emoji chooser: Match search terms better

Use g_str_match_string for better results.

cherry-pick of be2853e5deb60274e43352352de74dc458908fe2

https://gitlab.gnome.org/GNOME/gtk/issues/898

Merge branch 'wip/lantw/gtk3-use-dev-evdev-input-h-on-freebsd' into 'gtk-3-22'

wayland: Use dev/evdev/input.h on FreeBSD (GTK3)

See merge request GNOME/gtk!132

Update Finnish translation

wayland: Use dev/evdev/input.h on FreeBSD

The header linux/input.h used by GDK is specific to Linux. It is
possible to get a few Linux headers on FreeBSD by installing v4l_compat,
but it is usually better to use the one shipped with FreeBSD.

We prefer dev/evdev/input.h to linux/input.h here, so it will always use
dev/evdev/input.h on FreeBSD regardless of v4l_compat.

https://svnweb.freebsd.org/changeset/ports/465644

Merge branch '169-gtktextview-accesses-already-disposed-object-3-22' into 'gtk-3-22'

Resolve "GtkTextView accesses already disposed object" in 3.22

See merge request GNOME/gtk!110

Merge branch 'window-activate-grab-3-3' into 'gtk-3-22'

gdk: do not deactivate window on keyboard grabs

See merge request GNOME/gtk!127

gdk: do not deactivate surface on keyboard grabs

When pressing e.g. a window manager shortcut, which acquires keyboard grab,
Xorg would send FocusOut NotifyGrab then FocusIn NotifyUngrab.  Currently
gdk would then deactivate the current surface, which makes accessibility
screen readers think that we have switched to a non-accessible application
and came back again, and thus reannounce the application frame etc. which we
don't want when e.g. just raising volume.

And actually, receiving FocusOut NotifyGrab does not mean losing the
X focus, it only means an application aqcuired a grab, i.e. it is
temporarily stealing keyboard events. On Wayland, this isn't even
notified actually.

This commit makes gdk only deactivate surfaces when there was an actual
focus switch to another window, as determined by has_focus_window (instead
of just has_focus), which happens either normally through FocusOut with
NotifyNormal, or during grabs through FocusOut with NotifyWhileGrabbed.

Fixes #85

(cherry picked from commit 01455399e83a3dbafb0cdc6e12c7003b2f472a40)

gtkselection: ensure W32 functions used only for W32 displays (v2)

Another instance of a check needed for gdk_win32_selection_add_targets().

gtkselection: ensure W32 functions used only for W32 displays

gdk_win32_selection_add_targets() is for W32 displays only.

imcontextsimple: ensure W32 code only runs on W32 displays

gdk_win32_keymap_check_compose() shouldn't be called for
non-W32 displays (i.e. when using broadway or other backends
that could be made to run on Windows).

fishbowl: Port version from GTK 4

This version also merges widgetbowl into fishbowl.

Add _gtk_printer_get_hard_margins_for_paper_size()

to retreive paper size specific hard margins and use this
to set the hard margins in the print context.
(modified by Marek Kasik <mkasik@redhat.com>)

https://bugzilla.gnome.org/show_bug.cgi?id=686109

Update Spanish translation

testentrycompletion: Avoid a compiler cast warning

Instead of getting the model again but then forgetting to cast it to a
ListStore, just use the other variable store where we already did both.

Update Russian translation

wayland: Allow a NULL inhibitors hash table

The shortcuts inhibitors hash table is created when we create a
GdkWaylandWindow implementation for a GdkWindow, and it's destroyed once
we finalize the instance. The fake "root" window we create for the
Wayland display does not have a backing native window, so the shortcuts
inhibitors hash table is set to NULL; this causes a critical error
message when calling g_hash_table_destroy() on it. The finalization of
the root window happens when we close a display connection.

We should use g_clear_pointer(), instead, as it's NULL safe.

Without this change, the displayclose test fails, as all warnings are
considered fatal.

build: Do not dist generated files

We expect these files to be regenerated even when building GTK+ from a
release tarball, so there's no point in distributing them if they are
going to be ignored.

x11: Query whether we have GLX support

Epoxy 1.4 has new ad hoc API that we can use to check whether GLX is
available on the current system.

If we didn't use this API, we'd have to manually dlopen libGL (or its
equivalent on different OSes) and check if it had GLX symbols; since
Epoxy already does all of this internally, we can simply ask it instead.

https://bugzilla.gnome.org/show_bug.cgi?id=775279
(cherry picked from commit 02eb344950547c66a9096094f271b98e94614fcb)
Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>

Bump up the dependency of libepoxy

We're going to use API introduced in libepoxy 1.4 to check for GLX
availability.

https://bugzilla.gnome.org/show_bug.cgi?id=775279

themes: Fix a new comment

bah.

themes: Fix swapped borders on RTL PathBar buttons

.linked assumes the container is a GtkBox, which is documented as never
flipping children in RTL, so :first-child is always the left child, etc.
GtkBox does that by reordering its CSS nodes when the direction changes.

But most widgets don’t do that, so :first|last-child are 1st/last ADDED
and swap sides in RTL. GtkPathBar is so, and ignoring that in our themes
meant that in RTL, its left/right buttons got each other’s borders. Yuk!

This patch adds the groundwork for supporting widgets like that, via the
%linked_flippable placeholder, and applies that to override buttons in
  filechooser .path-bar.linked > button
so that the correct borders get applied to those buttons when using RTL.

Note that I select only PathBars within a FileChooser because we also
have NautilusPathBar, which also uses widget.path-bar – but *does* flip
its nodes for RTL already, so letting that get affected broke it again!

https://bugzilla.gnome.org/show_bug.cgi?id=772817

MenuButton: popover connects to menu_deactivate_cb

This was missed (even before my recent patches).

https://gitlab.gnome.org/GNOME/gtk/issues/199

MenuButton: Disconnect newly added signal handler

Otherwise, we do stuff we shouldn't, as the failing pipeline showed:
https://gitlab.gnome.org/GNOME/gtk/pipelines/9431

https://gitlab.gnome.org/GNOME/gtk/issues/199

MenuButton: Clarify NULL popup/over/menu/model doc

MenuButton: Drop ref to Popover on its ::destroy

Otherwise, if the Popover is destroyed before the MenuButton, the latter
still had a non-NULL but invalid instance and tried to use it in dispose

https://gitlab.gnome.org/GNOME/gtk/issues/199

docs/running: Document the GTK_OVERLAY_SCROLLING env var

https://bugzilla.gnome.org/show_bug.cgi?id=786524

Stock: Emphasise deprecation; explain alternatives

A user in #gtk+ was confused what to do instead of creating a Button via
gtk_button_new_from_stock(). Our docs could stand to be clearer on this
point; it only costs a few lines. So, link from that constructor* to the
GtkStock doc, and add a banner there telling folk they shouldn’t use it.

* not that most [of these][links] even work right now…

Entry: Show optional style classes in node diagram

gdkselection: Clarify that X is only a design base

rather than being a requirement for using our selection APIs.

https://bugzilla.gnome.org/show_bug.cgi?id=791542

a11y/ScrolledWin|IconView: Connect signals safely

Use g_signal_connect_data() instead of g_signal_connect_object()
to make sure the callback gets disconnected when the data object
is destroyed. This avoids problems in garbage-collected bindings.

https://bugzilla.gnome.org/show_bug.cgi?id=789215

printing: Be more careful when deserializing

The GVariant we are getting here might not be coming
from GTK+, but rather from some other source. Best to
be forgiving and deal with missing data without crashing.

This was causing the GTK+ portal backends to crash on
print requests from Qt.

Merge branch 'stack-set-name-guard' into 'gtk-3-22'

stack: protect set_visible_child_name from NULL stack

See merge request GNOME/gtk!117

Merge branch 'gtkplacesview-finalization-fixes-master' into 'master'

Gtkplacesview finalization fixes

See merge request GNOME/gtk!119

(cherry picked from commit e30176a5228228f28f914d87120b9a650d0073b8)

f9452957 gtkplacesview: unset entry_pulse_timeout_id before removing it
4900c3eb gtkplacesview: disconnect from server list monitor changes on destroy

stack: protect set_visible_child_name from NULL stack

Return with error if gtk_stack_set_visible_child_name is called
with NULL parameter

UIManager: Document deprecation; link replacements

Close https://gitlab.gnome.org/GNOME/gtk/issues/193

Range: Use should_invert_move() to scroll value

This fixes RTL and/or :inverted Ranges responding to a horizontal scroll
by moving the value/slider button in the opposite direction... See prev.

https://bugzilla.gnome.org/show_bug.cgi?id=791802

Range: Add should_invert_move() for scrolls & keys

This will be used in subsequent commits to fix the sign by which the
value is changed in response to directional scroll or keypress events.

The idea is: you have a movement to make – in the form of a delta that
follows widget directions, i.e. −1 means left or up, +1 means right or
down – and you want to know whether that delta needs to be inverted in
order to produce the intuitively expected directional change of :value.

The existing should_invert() is not sufficient: it just determines
whether to invert visually, but we need more nuance than that for input.

To answer that – while not doubling up the work for scrolls and keys – I
add a helper should_invert_move(), which considers other relevant state:

 • A parallel movement on priv->orientation should just use the existing
   should_invert(), which already worked OK for this case (not others).

 • Movements on the other orientation now depend on priv->orientation:

    ◦ For a horizontal Range, always invert, so up (i.e. −ve in terms of
      widget coords) always means increase value & vice-versa. This was
      done in get_wheel_delta(), but move it here for use with keys too.

    ◦ For a vertical Range, ignore :invert as it’s only relevant to the
      parallel orientation. Do not care about text direction here either
      as RTL locales do not invert number lines, Cartesian plots, etc.

This returns TRUE if the delta should be inverted before applying to the
value, and we can now use this function in both scroll and key handlers.

https://bugzilla.gnome.org/show_bug.cgi?id=407242
https://bugzilla.gnome.org/show_bug.cgi?id=791802

Menu|Item: Fix FIXME re non-const interned strings

These are members of the private struct, so it hurts no one to fix this.

Menu: Sanitise @torn_off gboolean in public API